Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add cotton, strings, and nets #658

Merged
merged 10 commits into from
Sep 4, 2022
Merged

Add cotton, strings, and nets #658

merged 10 commits into from
Sep 4, 2022

Conversation

byorgey
Copy link
Member

@byorgey byorgey commented Sep 1, 2022

  • Add a cotton entity that grows in some of the places there used to be flowers
  • 4 cotton make one string, with a small motor catalyst. Currently, string serves no purpose except being an ingredient for net, but I imagine we can find other fun things to do with strings.
  • 256 string make one net.
  • A net is a device that enables the try command.

This is intentionally a bit on the harder side to construct. You need to set up some good automation to be able to crank out the nets if you want to be able to use try on your robots.

Closes #132.

xsebek
xsebek approved these changes Sep 2, 2022
View reviewed changes
Copy link
Member

@xsebek xsebek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Maybe we could tie it to the string type. Otherwise it might be good to rename it to text.

@byorgey
Copy link
Member Author

byorgey commented Sep 2, 2022 •
edited
Loading

Hmm, what if string is a device that allows format and ++? Right now a logger is required to use those.

Maybe renaming the string type to text is a good idea anyway; the name string is a historical accident tied to a specific representation of text. Then the fact that a string is the device for doing stuff with text would itself be a kind of joke.

xsebek
xsebek reviewed Sep 2, 2022
View reviewed changes
data/scenarios/Testing/508-capability-subset.yaml
- 3D printer
inventory:
- [1, mirror]
- [1, string]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😉

@byorgey byorgey added the merge me Trigger the merge process of the Pull request. label Sep 4, 2022
@restyled-io restyled-io bot mentioned this pull request Sep 4, 2022
Merged
@restyled-io @restyled-commits
Restyled by fourmolu (#661)
5f8feef 
Co-authored-by: Restyled.io <commits@restyled.io>
@mergify mergify bot merged commit b3651e2 into main Sep 4, 2022
@mergify mergify bot deleted the cotton branch September 4, 2022 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xsebek xsebek xsebek approved these changes

Assignees
No one assigned
Labels
merge me Trigger the merge process of the Pull request.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Device to give exceptions capability
2 participants
@byorgey @xsebek